x11: Remove some leftover event struct access

x11: Stop using the send_event event field

The only use of this field is printing it out in a
debug message. We are going to drop it.

Always deliver focus events to toplevels

Its was GTK expects. This change gets rid of the "Ignoring an
unexpected focus event from GDK on a non-toplevel surface."
warning.

Drop gtk_widget_event from API

We don't want events to be injected randomly from
the outside. Plus, there's no way to create such
events from the outside now.

events: reorganize getters

Restructure the getters for event fields to
be more targeted at particular event types.

Update all callers, and replace all direct
event struct access with getters.

As a side-effect, this drops some unused getters.

Drop gdk_set/get_show_events

We don't need a separate api to turn on debugging
for events.

Strip const from GdkEvent

Events are refcounted structs, and we generally don't
pass these as const.

events: Drop all setters

GdkEvent is now truly readonly.

events: Drop target and related target

GTK no longer uses this, so we can drop it.

main: Stop calling gdk_event_set_target

GTK is no longer relying on this.

Stop using gdk_event_get_target

We can now get the target widget from the gesture
that we are using to find the event in the first
place.

gesture: Keep the target widget for events

We already store the events; keep the target widget
in addition. This is a step towards getting rid of
gdk_event_get_target.

eventcontroller: Make the target widget available

Make it possible for event controllers to obtain
the target widget during handle_event.

Explicitly pass the target to handle_event

Pass the event propagation target explicitly down to
the event controllers. This is a step towards getting
rid of gdk_event_set_target.

Stop looking at the related target for filtering

We are now sending crossing events (which are the only ones
where a related target makes sense) via handle_crossing.

Reinstate filtering for crossing events

The event propagation limit should apply to crossing events
as well.

Make crossing events handled the same way

New focus change handling

Instead of relying on gdk's antiquated crossing events,
create a new GtkCrossingData struct that contains the
actual widgets, and a new event controller vfunc that
expects this struct. This also saves us from making sense
of X's crossing modes and details, and makes for a
generally simpler api.

The ::focus-in and ::focus-out signals of GtkEventControllerKey
have been replaced by a single ::focus-change signal that
takes GtkCrossingData as an argument. All callers have
been updated.

wip: scrolledwindow stop using targets

main: Drop gtk_get_event_target

This is no longer used.

tooltip: stop using gtk_get_event_target

Stop using gtk_get_event_target

This is just a thin wrapper around gdk_event_get_target,
so use that directly.

Pass translated coordinates outside the event

We want to make events readonly, so stop translating
their coordinates and instead pass the translated
coordinates separately, when propagating events.

Drop GDK_NOTHING

Events of type GDK_NOTHING are good for nothing.

Make GdkEvent a boxed type

Stop using g_object_ref/unref on events

Use gdk_event_ref/unref instead of g_object_ref/unref.
Events will stop being object soon.

Drop gdk_event_new and gdk_event_copy

These functions are no longer used outside of gdkevents.c.

display: Stop using gdk_event_copy

Events are effectively readonly in GDK now, so we can just
take a reference, no need for a copy.

win32: Use event constructors

x11: Pass the right surface to the dnd filter

x11: Remove an unnecessary check

We are not creating GDK_NOTHING events anymore. Yay

x11: change event translator interface

Make the event translator return a new event, instead of
filling in a half-constructed one.

Update the two implementation in GdkX11Display and
GdkDeviceManagerXI2.

x11: Change the wm protocols filter api

Instead of passing a half-constructed event and expect
it to be filled in, pass the surface as in argument, and
add an out argument for a newly constructed GdkEvent.

x11: Don't pass a GdkEvent to shape cache filters

The filter functions never look at that event, and
we want to get out of the business of passing half-
constructed events around.

x11: Don't pass a GdkEvent to xsettings filters

The filter functions never look at that event, and
we want to get out of the business of passing half-
constructed events around.

events: Drop GDK_DESTROY

No backend is emitting GDK_DESTROY events anymore, so no
need to carry this around.

gtk: Stop handling GDK_DESTROY differently from GDK_DELETE

We don't have child windows anymore, so there is no difference.

x11: Don't generate GDK_DESTROY events

GTK does not differentiate between GDK_DESTROY and GDK_DELETE
anyway.

x11: use event constructors

This is not quite complete and only handles the
simple cases.

broadway: Use event constructors

Convert all of Broadways event handling to use the new
constructors.

wip: Use event constructors in the testsuite

This doesn't work because the constructors aren't exported,
so remove the gestures text temporarily, until we figure
out how to deal with it.

textview: Stop using gdk_event_copy

main: Use constructors instead of gdk_event_copy + rewriting

imcontextsimple: Use event constructors

widget: Port emulation code to constructors

Use the new event constructors when generating events
for emulating presses.

window: Use event constructors

main: Use event constructors

At the same time, stop setting child_surface in
crossing events. Nothing in GTK looks at it.

gdk: Use constructor for grab broken

gdk: Use event constructors for dnd

events: Drop x_root/y_root

We are not using these fields anymore.

broadway: Stop setting x_root/y_root in events

These fields are not used in GTK anymore, and are
going away soon.

x11: Stop setting x_root/y_root in events

These fields are not used by GTK anymore and
will be going away soon.

x11: Stop relying on root coordinates in events

The X11 dnd code was still using x_root/y_root in
GdkEventMotion in a couple of places. Stop doing so,
these field are going away soon.

Remove root coords from the GdkDrop api

wayland: Clean up coordinate handling

Get rid of the get_coordinates function and just
use the structs directly.

wayland: Use event constructors

Convert all of Waylands event handling to use the new
constructors.

gdk: Add event constructors

Add private API to construct events. This is a step towards
making events readonly, and not objects anymore.

The constructors here are sufficient to convert the Wayland
backend over. More may be added for other backends as needed.

Open issues:
 - history

Make a11y testsuite pass

Merge branch 'master' into 'master'

build: gdk/wayland/cursor depends on wayland-client

See merge request GNOME/gtk!1453

build: gdk/wayland/cursor depends on wayland-client

On distros where wayland-client.h might not be directly in /usr/include
we fail to find the correct headers otherwise.

scale: Remove useless extra local variable

scale: Remove unused member

emojichooser: Add style class to emoji toolbar

Closes #1013

filechooserwidget: Fold function into only caller

filechooserwidget: Use a proper GtkPopoverMenu

Instead of a fake one where we add .menu to a normal popover. This makes
the menu look like other context menus.

filechoosererrorstack: Set a layout manager

Otherwise it doesn't know what to do with its child widget.

filechooserwidget: Allow typing a location in recent mode

This is triggered by typing / or hitting Ctrl+L. Since we don't have a
visual indicator for this mode right now anyway, the reason for not
allowing it in recent mode cited in the comment just above the
early-exit is irrelevant.

Closes #2178

filechooserwidget: Remove unused constants

text: Remove gtk_text_get_im_context()

Unused.

docs: Remove focus_tracking.txt

All the info in there is outdated and not useful.

docs: Remove developers.txt

Everything in there has either been removed in the past or is outdated
information.

docs: Remove dnd_internals.txt

This is either already completely outdated or will be soon.

spinbutton: Remove manual queue_draw() calls

Everything is automatically redrawn when changing the value, etc.

popvermenu: Clarify css node docs

We add the .menu css class to the popover main node, not its contents
node.

aboutdialog: Clarify docs and fix preconditions

modelbutton: Un-select on mouse leave

This brings us closer to the old GtkMenuItem behavior and also makes
more sense.

aboutdialog: Replace visited_links GList with GPtrArray

aboutdialog: Use new GtkStackPage API

stack: Add GtkStackPage:visible accessors

Merge branch 'wip/xdg-popup-async-relayout' into 'master'

Synchronous popup layout

See merge request GNOME/gtk!1241

Merge branch 'ci-flatpak' into 'master'

ci: Update flatpak jobs

See merge request GNOME/gtk!1451

ci: Update flatpak jobs

The CI infrastructure was changed to run with fewer privileges, so we
need to adapt our Flatpak jobs to compensate.

See: https://mail.gnome.org/archives/desktop-devel-list/2020-February/msg00055.html

gdk/surface: Replace move_to_rect() with GdkPopupLayout based API

Replace the gdk_surface_move_to_rect() API with a new GdkSurface
method called gdk_surface_present_popup() taking a new GdkPopupLayout
object describing how they should be laid out on screen.

The layout properties provided are the same as the ones used with
gdk_surface_move_to_rect(), except they are now set up using
GdkPopupLayout.

Calling gdk_surface_present_popup() will either show the popup at the
position described using the popup layout object and a new unconstrained
size, or reposition it accordingly.

In some situations, such as when a popup is set to autohide, presenting
may immediately fail, in case the grab was not granted by the display
server.

After a successful present, the result of the layout can be queried
using the following methods:

 * gdk_surface_get_position() - to get the position relative to its
   parent
 * gdk_surface_get_width() - to get the current width
 * gdk_surface_get_height() - to get the current height
 * gdk_surface_get_rect_anchor() - to get the anchor point on the anchor
   rectangle the popup was effectively positioned against given
   constraints defined by the environment and the layout rules provided
   via GdkPopupLayout.
 * gdk_surface_get_surface_anchor() - the same as the one above but for
   the surface anchor.

A new signal replaces the old "moved-to-rect" one -
"popup-layout-changed". However, it is only intended to be emitted when
the layout changes implicitly by the windowing system, for example if
the monitor resolution changed, or the parent window moved.

gdk/surface: Reset position after hiding

The x and y position of GdkSurface is only valid when visible, so reset
them to their inital state after hiding.

wayland: Simplify logic deciding whether to use xdg_popup

From a handful of hueristics and guess-work, to a single
`surface->surface_type == GDK_SURFACE_POPUP`.

wayland: Fix top-most-popup check

We can map a non-grabbing popup wherever, it's just the grabbing
popup-chain that needs to be ensured not to break any ordering rules.

Fix this by managing two lists; one of open popups, and another for
grabbing ones.

gdk: Move GdkGeometry declaration to gdktypes.h

gdk: Move GdkGravity to gdktypes.h

x11: Make gdk_surface_get_position() return relative position

The returned position should be relative to the parent surface, but
GdkSurface::x,y were only managed properly for O-R windows. This makes
it correct for regular windows too.

wayland: Simplify popup parent discovery

Now popups surfaces are always created with the parent set, so we don't
need to implement vorious guess work to try to find what the parent
might be. Remove that code and just use GdkSurface::parent which is
where the parent set during construction ends up at.

wayland: Remove old "gdk-attached-grab-surface" hackery

The corresponding code in gtk/ is long gone, so it's luckly unused these
days.

wayland/surface: Add per surface configuration event queues

Add event queues specifically for surface configuration events
(xdg_surface.configure, xdg_toplevel.configure, xdg_popup.configure etc)
so that a configuration can be completed without having side effects on
other surfaces. This will be used to synchronously configure specific
GdkSurfaces, as is needed by the Gtk layout mechanisms.

gdk/wayland/surface: Use dedicated bool for frame callback freezing

The freezing is conditioned on various state, so lets make the thawing a
bit more robust. Without this there was a risk that we'd thaw too many
times if there was a frame callback requested while the conditions for
the freezing were not met.

wayland: Remove position method split

Position can only be done via gdk_surface_queue_relayout(); the legacy
way is no longer possible, so remove the legacy positioning code.

gdk/surface: Use enum type for surface type

tests/popover: Add check box for switching autohide prop

popover: Unrealize when autohide prop changes

Changing the autohide property means the popover needs to be remapped.
Remapping may need user interaction, so lets just unrealize the popover
if the property changes, forcing the application to remap it should it
be shown again.

Merge branch 'file-filter-name-property' into 'master'

GtkFileFilter: Add a GObject property for the name

See merge request GNOME/gtk!376

Merge branch 'icontheme-api' into 'master'

Icontheme api

Closes #2410

See merge request GNOME/gtk!1449

icontheme: Update the api

Add properties, and use string arrays instead of lists.

Among other things, this renames gtk_icon_theme_list_icons
to gtk_icon_theme_get_icon_names.

Fixes: https://gitlab.gnome.org/GNOME/gtk/issues/2410